fix: update remaining opencode references in help output and CLI#547
fix: update remaining opencode references in help output and CLI#547VJ-yadav wants to merge 1 commit intoAltimateAI:mainfrom
Conversation
|
This PR doesn't fully meet our contributing guidelines and PR template. What needs to be fixed:
Please edit this PR description to address the above within 2 hours, or it will be automatically closed. If you believe this was flagged incorrectly, please let a maintainer know. |
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 29 minutes and 53 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (5)
📝 WalkthroughWalkthroughThis PR updates branding and runtime targets from "opencode" to "altimate-code": CLI PR command now imports/spawns Changes
Sequence Diagram(s)sequenceDiagram
participant User as "User"
participant CLI as "opencode CLI\n(packages/opencode/src/cli/cmd/pr.ts)"
participant Importer as "altimate-code\n(import subcommand)"
participant TUI as "altimate-code TUI\n(spawned process)"
rect rgba(200,220,255,0.5)
User->>CLI: run `pr` command (with PR URL)
CLI->>Importer: Process.text(["altimate-code","import", sessionUrl])
Importer-->>CLI: stdout includes sessionId
CLI->>TUI: spawn("altimate-code", ["tui","--session", sessionId])
TUI-->>User: TUI UI appears
end
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
377e484 to
ade8d48
Compare
❌ Tests — Failures DetectedTypeScript — 15 failure(s)
cc @VJ-yadav |
Review — Ready for mergeScope: Fixes #416 — replaces user-facing Analysis
Files changed (6): CI status (commit
Recommendation cc @anandgupta42 — ready for |
Replace user-visible "opencode" branding in CLI --help descriptions, mDNS defaults, and pr command binary spawns with "altimate-code". Closes AltimateAI#417, relates to AltimateAI#416. Co-Authored-By: Vijay Yadav <vijay@studentsucceed.com>
92424f7 to
27e9fd4
Compare
Consensus code-review applied — no changes neededRan a 4-participant consensus review (Claude + GPT 5.4 + Gemini 3.1 Pro + self-review). Verdict: APPROVE — mostly user-facing branding + one functional fix ( GPT flagged two MAJOR concerns that I investigated and assessed as false positives:
The intentionally-preserved opencode references ( All CI green. Ready for merge as-is. |
Summary
Fixes #416 — Updates user-facing
opencodereferences toaltimate-codeacross help text, CLI commands, and mDNS discovery.Changed (8 files, 19 replacements):
thread.ts— TUI help descriptionsuninstall.ts— uninstall command descriptionpr.ts— user-facing messages +spawn("opencode", ...)→spawn("altimate-code", ...)(functional fix — old binary name fails on rebranded installs)network.ts,config.ts,mdns.ts— mDNS domainopencode.local→altimate-code.localthread.test.ts— test expectations for mDNS domaintypes.gen.ts— JSDoc mDNS domain referenceIntentionally NOT changed: Internal identifiers (
@opencode/Auth,ProviderID.opencode,OPENCODE_*env vars),@opencode-ai/*package names,.opencodeconfig directories, npm registry names in uninstall,User-Agent/X-Titleprotocol headers.Test Plan
bun turbo typecheckpasses across all 5 packagesbun test test/cli/tui/thread.test.tspasses with updated mDNS expectationsaltimate-code --helpshows noopencodein descriptionsaltimate-code prmessages referencealtimate-codeChecklist
Summary by CodeRabbit
Chores
Tests